Skip to content

Conversation

@jaydeluca
Copy link
Member

@jaydeluca jaydeluca commented Nov 10, 2025

Disabling the JSP compile spans by default. They can be re-enabled using the otel.instrumentation.common.experimental.view-telemetry.enabled=true configuration flag

Metadata updates are part of #14096 and #14128

@jaydeluca jaydeluca requested a review from a team as a code owner November 10, 2025 15:09
- name: jsp-2.3
display_name: JSP (JavaServer Pages)
description: |
This instrumentation enables view spans for JSP page rendering and compilation (view spans are disabled by default).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@trask the jsp compile spans are enabled by default. Should we disable them?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@jaydeluca jaydeluca changed the title JSP Metadata Disable JSP compile spans by default Nov 17, 2025
@github-actions
Copy link
Contributor

⚠️ Breaking Change Documentation Required

This PR has been labeled as a breaking change. Please ensure you provide the following information:

Migration Notes Required

Please add detailed migration notes to help users understand:

  • What is changing and why
  • How to update their code/configuration
  • Any alternative approaches if applicable
  • Code examples showing before/after usage

Checklist

  • Migration notes added to the PR description
  • Breaking change is documented in the changelog entry for the next release
  • Consider if this change requires a major version bump

Your migration notes will be included in the release notes to help users upgrade smoothly. The more detailed and helpful they are, the better the user experience will be.


This comment was automatically generated because the breaking change label was applied to this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants